Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Oh-x-accounts #65

Closed
wants to merge 3 commits into from
Closed

Oh-x-accounts #65

wants to merge 3 commits into from

Conversation

akaneshiro7
Copy link
Contributor

What was the ticket?

OH-X-ACCOUNTS
Fix styling on accounts page and

Link to Ticket: #64

What did I do?

Make one page
fix side image
fix header
add state

How did I test it? Document test cases here.

look at figma, look at localhost:3000/accounts
console.log(state) when updating

What could go wrong in the future? What parts of your code should the reviewer pay the most attention to?

Create User endpoint might need more parameters than email/password when sending request on Sign Up button.

Screenshots

image

@akaneshiro7 akaneshiro7 added the closes issue This change closes an open issue label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closes issue This change closes an open issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant